Skip to content
This repository was archived by the owner on Aug 17, 2023. It is now read-only.

#215 blackを導入 - #217

Open
kemu3007 wants to merge 2 commits into
masterfrom
t215_add_black
Open

#215 blackを導入#217
kemu3007 wants to merge 2 commits into
masterfrom
t215_add_black

Conversation

@kemu3007

@kemu3007 kemu3007 commented Nov 2, 2020

Copy link
Copy Markdown

チケットURL

このレビューで確認してほしい点

  • blackを用いたformat, ciでのテストが走るようになっている

レビューチェックリスト

  • C2 体を表す名前の公理:あらかじめ決められている以外の汎用的な名前のモジュールを作らない
  • C3 汎用名のモジュール内に長々と具体的処理を書かない
  • C4 単純な処理の長さで分割しない
  • C5 引数の数を減らす
  • C6 パッケージ間で共通した定数を作らない
  • C7 継承の利用を最小限にする
  • C8 親クラスのテストを子クラスでも実行すること
  • C9 オーバーライドを減らす
  • C10 継承やオーバーライドを明示する

@kemu3007 kemu3007 self-assigned this Nov 2, 2020
Comment thread tox.ini

[testenv:flake8]
[testenv:lint]
deps =

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lint, fmt, lint-ciコマンドを追加しました、これまで利用していたflake8, flake8_ciはlint, lint_ciに統合されています

@kemu3007
kemu3007 requested review from a team and jamesvandyne and removed request for a team November 2, 2020 07:38

@jamesvandyne jamesvandyne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants